Skip to content

Conversation

@mwtian
Copy link
Contributor

@mwtian mwtian commented Dec 2, 2025

Description

When proposing a block, only votes on blocks within DagState GC round at proposal time are included.
So when considering votes in CommitFinalizer, if GC round is <= R when proposing block B, any block with round <= R should not receive votes from B.

Test plan

CI
antithesis

@mwtian mwtian requested a review from a team as a code owner December 2, 2025 22:26
@vercel
Copy link

vercel bot commented Dec 2, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
sui-docs Ready Ready Preview Comment Dec 4, 2025 10:10pm
2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
multisig-toolkit Ignored Ignored Preview Dec 4, 2025 10:10pm
sui-kiosk Ignored Ignored Preview Dec 4, 2025 10:10pm

@mwtian mwtian temporarily deployed to sui-typescript-aws-kms-test-env December 2, 2025 22:26 — with GitHub Actions Inactive
@mwtian mwtian temporarily deployed to sui-typescript-aws-kms-test-env December 3, 2025 05:41 — with GitHub Actions Inactive
@mwtian mwtian requested a review from mystenmark as a code owner December 4, 2025 09:00
@mwtian mwtian temporarily deployed to sui-typescript-aws-kms-test-env December 4, 2025 09:00 — with GitHub Actions Inactive
@mwtian mwtian temporarily deployed to sui-typescript-aws-kms-test-env December 4, 2025 09:01 — with GitHub Actions Inactive
@mwtian mwtian changed the title [mfp] use more conservative filter for transaction votes [mfp] do not vote on potentially GC'ed blocks Dec 4, 2025
@mwtian mwtian temporarily deployed to sui-typescript-aws-kms-test-env December 4, 2025 09:03 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants